Bug fix for pipeline downloading incorrect package ver #20294
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ckairen
requested review from
danieljurek,
scbedd and
weshaggard
as code owners
weshaggard
reviewed
Aug 16, 2021
weshaggard
reviewed
Aug 16, 2021
| @@ -185,7 +185,22 @@ jobs: | |||
| env: | |||
| REQUIREMENTS: $(requirements) | |||
|
|
|||
| - pwsh: pip install -r "$env:REQUIREMENTS" --no-deps --upgrade | |||
| - pwsh: | | |||
| $timeout = New-TimeSpan -Seconds 600 | |||
weshaggard
reviewed
Aug 16, 2021
weshaggard
reviewed
Aug 16, 2021
weshaggard
reviewed
Aug 17, 2021
weshaggard
reviewed
Aug 17, 2021
weshaggard
reviewed
Aug 17, 2021
weshaggard
reviewed
Aug 17, 2021
weshaggard
reviewed
Aug 18, 2021
weshaggard
reviewed
Aug 18, 2021
weshaggard
reviewed
Aug 18, 2021
weshaggard
reviewed
Aug 18, 2021
benbp
reviewed
Aug 19, 2021
benbp
reviewed
Aug 19, 2021
benbp
reviewed
Aug 19, 2021
|
/azp run python - smoke-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
weshaggard
reviewed
Aug 19, 2021
| @@ -159,15 +159,15 @@ jobs: | |||
| timeoutInMinutes: 5 | |||
|
|
|||
| - pwsh: | | |||
| $packages = Get-ChildItem "$(Pipeline.Workspace)/${{ parameters.ArtifactName }}/*.zip" | |||
| $packages = Get-ChildItem "$(Pipeline.Workspace)/${{ parameters.ArtifactName }}/${{ parameters.Artifact.name }}/*.zip" | |||
| Write-Host "Artifacts found:" | |||
There was a problem hiding this comment.
We should still try to add some sort of validation to make sure we found the packages we were expecting.
There was a problem hiding this comment.
Isn't this essentially just testing that our release build scripts work and build the right thing? I think that might be an environment assumption we can make and let the build scripts be responsible for validating that? Or am I misunderstanding.
There was a problem hiding this comment.
I just want to make sure we don't accidently slip through this step and not update any package references in the requirements and then everything works as expected but we didn't actually test anything.
There was a problem hiding this comment.
Makes sense. Do you happen to know of a clear reference to the current version being released? It seems like we may have to export a variable as part of our version update scripts the way we did for java?
There was a problem hiding this comment.
I think for our purposes here the goal is to test the artifact we have just built so we can continue to parse the version from the artifact if we want (or we can switch to getting that from the PackageInfo.json) and then we just need to check to make sure what we passed in via the parameter.Artifact matches what we found and are updating.
There was a problem hiding this comment.
@ckairen @scbedd and I are still a little confused as to what's being proposed here. Within the release pipeline context, based on previous steps and the actions of the release engineer, the build artifact and version in it is a source of truth for the version. So if we wanted to check that we were pulling in the right versioned artifact, wouldn't we be essentially be checking the same source of version data against itself?
Since you're out until next week, I think we should merge what's here to fix the immediate issue, but keep #19928 open so we can discuss/clarify when you get back.
There was a problem hiding this comment.
I reopened #19928 and added some more context there.
benbp
approved these changes
Aug 20, 2021
|
/azp run python - smoke-tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
scbedd
approved these changes
Aug 20, 2021
scbedd
approved these changes
Aug 20, 2021
iscai-msft
added a commit
to iscai-msft/azure-sdk-for-python
that referenced
this pull request
Aug 23, 2021
…into have_pipelines_support_rest * 'main' of https://github.com/Azure/azure-sdk-for-python: (27 commits) Document Pod Identity's special use of client_id (Azure#20377) [rest] use azure json encoder for json input bodies (Azure#20361) CertificateCredential supports PKCS12 certs (Azure#16384) Bug fix for pipeline downloading incorrect package ver (Azure#20294) Update metadata values (Azure#20365) [Key Vault] Add 7.3-preview support for administration (Azure#20364) Fix Monitor opentelemetry exporter readme issues (Azure#19038) More Renaming in query (Azure#20303) Update CODEOWNERS (Azure#20366) [ServiceBus] update migration guide with message count info (Azure#20360) [rest] change text from a property to a method (Azure#20290) Handle value types for results (Azure#20358) Remove old unused update changelog script (Azure#20357) bump node version (Azure#20353) [AutoRelease] t2-web-2021-08-03-73015 (Azure#20053) Fix query batch processing (Azure#20345) Sync eng/common directory with azure-sdk-tools for PR 1912 (Azure#20340) Increment version for schemaregistry releases (Azure#20326) [SchemaRegistry] prepare avro for release (Azure#20321) Update CHANGELOG.md (Azure#20334) ...
hildurhodd
pushed a commit
to hildurhodd/azure-sdk-for-python
that referenced
this pull request
Aug 26, 2021
* Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version
hildurhodd
pushed a commit
to hildurhodd/azure-sdk-for-python
that referenced
this pull request
Aug 26, 2021
* Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version
hildurhodd
pushed a commit
to hildurhodd/azure-sdk-for-python
that referenced
this pull request
Aug 30, 2021
* Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version
msyyc
added a commit
that referenced
this pull request
Aug 30, 2021
* CodeGen from PR 15691 in Azure/azure-rest-api-specs [NetAppFiles] fix default in volumeproperties (#15691) * fix default in volumeproperties * Update description * Update description Co-authored-by: Audunn Baldvinsson <[email protected]> * version,CHANGELOG * test * bump node version (#20353) * Remove old unused update changelog script (#20357) Co-authored-by: Wes Haggard <[email protected]> * Handle value types for results (#20358) * Handle value types for results * update test * lint * comprehension * more precis * fix test * [rest] change text from a property to a method (#20290) * [ServiceBus] update migration guide with message count info (#20360) #20245 * Update CODEOWNERS (#20366) * More Renaming in query (#20303) * More Reanaming in query * changelog * commit 2 * some changes * remove errror * Update sdk/monitor/azure-monitor-query/azure/monitor/query/_models.py * Apply suggestions from code review Co-authored-by: Adam Ling (MSFT) <[email protected]> * Update sdk/monitor/azure-monitor-query/CHANGELOG.md Co-authored-by: Adam Ling (MSFT) <[email protected]> * Fix Monitor opentelemetry exporter readme issues (#19038) * [Key Vault] Add 7.3-preview support for administration (#20364) * Update metadata values (#20365) * Update metadata values * distributed trace * oops * Update sdk/monitor/azure-monitor-query/CHANGELOG.md * Apply suggestions from code review * Bug fix for pipeline downloading incorrect package ver (#20294) * Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version * CertificateCredential supports PKCS12 certs (#16384) * [rest] use azure json encoder for json input bodies (#20361) * Document Pod Identity's special use of client_id (#20377) * Consistency related changes (#20385) * time stamp * Rename AggragationType to MetricAggregationType * Metric Class * logs batch result * MetricNamespaceClassification * LogsTable + LogsTableColumn * lint * more lint * Add Cloud Configuration section to Identity readme (#20373) * keyvault and storage have a conflict between mindependency and what local azure-identity requires. resolve it (#20391) * [AutoRelease] t2-costmanagement-2021-07-27-84467 (#19949) * CodeGen from PR 15291 in Azure/azure-rest-api-specs Updating Query filter description (#15291) Co-authored-by: Jorge Chavez Nieto <[email protected]> * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Jorge Chavez Nieto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * [AutoRelease] t2-servicebus-2021-08-19-00756 (#20350) * CodeGen from PR 14749 in Azure/azure-rest-api-specs add apitestErrorCode doc (#14749) * add apitestErrorCode doc * add armRPC doc * add roundTripInconsistentProperty doc * small fix * update * update signalR armTemplate * update yaml * new file * add testScenario with armTemplate * update doc * update readme.md * update doc * update NOTE section * add signalRCreateOrUpdate example file * udpate doc * add generate test scenario section * update doc * add serviceFacbric test scenario file * update managedClusters.yaml * update generateABasicTestScenario.md * update features doc * add run api test gif * version,CHANGELOG * test * Update test_cli_mgmt_servicebus_namespace.py * Update test_queues.py * Update test_queues.py Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: Zed Lei <[email protected]> Co-authored-by: Jiefeng Chen <[email protected]> * [AutoRelease] t2-policyinsights-2021-08-23-78735 (#20381) * CodeGen from PR 15165 in Azure/azure-rest-api-specs [policyinsights] Fix next_link (#15165) * fix next_link * directive * Update readme.nodejs.md * js directvie * Update readme.md * supress other language * Update readme.go.md * swagger lint * swagger lint * swagger lint * Update readme.md * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: Zed Lei <[email protected]> * [AutoRelease] t2-datamigration-2021-08-19-04035 (#20349) * CodeGen from PR 15259 in Azure/azure-rest-api-specs Dev datamigration microsoft.data migration 2021 06 30 (#15259) * Created new api version and refactoring changes (such as fixing new build restrictions) * Adding breaking changes * Adding location to the services update example * Adding location for projects update and refactoring the task command * Resolving tasks command ModelValidation test failure * Resolving issue based on zhenglaizhang's recommendation * Adding comma to resolve issue Co-authored-by: Artem Pavlichenko <[email protected]> * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Artem Pavlichenko <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * Chain exceptions from LibsecretPersistence (#20380) * [translation] fix poller.details (#20392) * fixing docstring types for async client * populate poller.details.id immediately * pylint * add bug fix to changelog * improve begin_translation docstring (#20401) * Update .docsettings.yml (#20402) * Sync eng/common directory with azure-sdk-tools for PR 1918 (#20342) * Stress test deployment script resiliency fixes * Use PSModule-Helpers to install powershell-yaml Co-authored-by: Ben Broderick Phillips <[email protected]> * improve docs and samples for glossaries and custom models (#18587) * update the readme * update readme file * added custom translation samples * fix 'no-locale' thing in links * update glossary docs * update glossaries * link to sample glossaries instead of writing code in readme * update custom model sample linking * remove relative linking in readme * make subheadings in bold text to be more readable * conform with 'Document Translation' naming * disambiguate container sas url * capitaliz Azure name * remove misplaced period * update samples -> custom model * update async sample -> custom model * remove localization from url * update readme with new file types for glossaries * adding sample glossaries -> xlf * white space * use simplified single input method * update 'job' terminology * update azure-core naming * update glossary blob file reference name * link to supported glossaries table * remove locale from url * [purview account] first release for azure-purview-account (#20335) * purview account * CI * verion-tolerant * endpoint order * add test for collections * Update CHANGELOG.md * review * update reamdme * skip custom translation model samples (#20414) * Address API view issues (#20407) * logs client * lc-2 * models changes * more changes * more changes * more arch changes * changelog * tests * lint * fix tests * timespan * [translation] naming alignment (#20406) * list_all_translation_statuses -> list_translation_statuses * list_all_document_statuses -> list_document_statuses * cancelled -> canceled, cancelling -> canceling * FileFormat -> DocumentTranslationFileFormat * update changelog * make core tests offline (#19986) * make core tests offline * clean up * use localhost instead of 127.0.0.1 * update * update * update * Get rid of LogsBatchQueryResult (#20418) * Get rid of LogsBatchQueryResult * Update sdk/monitor/azure-monitor-query/README.md * oops * comments * [AutoRelease] t2-iothub-2021-08-25-25696 (#20409) * CodeGen from PR 15722 in Azure/azure-rest-api-specs Removing readonly for isVerified property as it is no more readonly property (#15722) * version,CHANGELOG * test Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * [AutoRelease] t2-healthcareapis-2021-08-26-27542 (#20422) * CodeGen from PR 15381 in Azure/azure-rest-api-specs Update readme.md (#15381) * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * Stip empty changelog sections before release (#20437) Co-authored-by: Chidozie Ononiwu <[email protected]> * Update question-answering readme links (#20439) * ensure test principal creation succeeds properly (#20446) Co-authored-by: scbedd <[email protected]> * New dummy password in README (#20328) * Sync eng/common directory with azure-sdk-tools for PR 1943 (#20450) * Bug fix: Storing .env file for bicep file at service dir * Styling Co-authored-by: Albert Cheng <[email protected]> * [AutoRelease] t2-keyvault-2021-08-26-55443 (#20423) * CodeGen from PR 15458 in Azure/azure-rest-api-specs Add rotation permission (#15458) * Add rotation permission * Update keys.json Remove required from lifetimeActions * Change to expiryTime * version,CHANGELOG * test Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * new recordings Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Audunn Baldvinsson <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: iscai-msft <[email protected]> Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Rakshith Bhyravabhotla <[email protected]> Co-authored-by: swathipil <[email protected]> Co-authored-by: McCoy Patiño <[email protected]> Co-authored-by: Adam Ling (MSFT) <[email protected]> Co-authored-by: Tong Xu (MSFT) <[email protected]> Co-authored-by: ckairen <[email protected]> Co-authored-by: Charles Lowell <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Azure CLI Bot <[email protected]> Co-authored-by: Jorge Chavez Nieto <[email protected]> Co-authored-by: Zed Lei <[email protected]> Co-authored-by: Jiefeng Chen <[email protected]> Co-authored-by: Artem Pavlichenko <[email protected]> Co-authored-by: Krista Pratico <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Mohamed Shaban <[email protected]> Co-authored-by: msyyc <[email protected]> Co-authored-by: Xiang Yan <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: annatisch <[email protected]> Co-authored-by: Albert Cheng <[email protected]>
iscai-msft
pushed a commit
that referenced
this pull request
Sep 29, 2021
* Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version
iscai-msft
added a commit
that referenced
this pull request
Sep 29, 2021
* CodeGen from PR 15691 in Azure/azure-rest-api-specs [NetAppFiles] fix default in volumeproperties (#15691) * fix default in volumeproperties * Update description * Update description Co-authored-by: Audunn Baldvinsson <[email protected]> * version,CHANGELOG * test * bump node version (#20353) * Remove old unused update changelog script (#20357) Co-authored-by: Wes Haggard <[email protected]> * Handle value types for results (#20358) * Handle value types for results * update test * lint * comprehension * more precis * fix test * [rest] change text from a property to a method (#20290) * [ServiceBus] update migration guide with message count info (#20360) #20245 * Update CODEOWNERS (#20366) * More Renaming in query (#20303) * More Reanaming in query * changelog * commit 2 * some changes * remove errror * Update sdk/monitor/azure-monitor-query/azure/monitor/query/_models.py * Apply suggestions from code review Co-authored-by: Adam Ling (MSFT) <[email protected]> * Update sdk/monitor/azure-monitor-query/CHANGELOG.md Co-authored-by: Adam Ling (MSFT) <[email protected]> * Fix Monitor opentelemetry exporter readme issues (#19038) * [Key Vault] Add 7.3-preview support for administration (#20364) * Update metadata values (#20365) * Update metadata values * distributed trace * oops * Update sdk/monitor/azure-monitor-query/CHANGELOG.md * Apply suggestions from code review * Bug fix for pipeline downloading incorrect package ver (#20294) * Bug fix: pipeline to download correct package ver * Update clean version * Update clean version * Cleanup * Cleanup * Cleanup * Cleanup * Comments added * Testing * Final-Version * CertificateCredential supports PKCS12 certs (#16384) * [rest] use azure json encoder for json input bodies (#20361) * Document Pod Identity's special use of client_id (#20377) * Consistency related changes (#20385) * time stamp * Rename AggragationType to MetricAggregationType * Metric Class * logs batch result * MetricNamespaceClassification * LogsTable + LogsTableColumn * lint * more lint * Add Cloud Configuration section to Identity readme (#20373) * keyvault and storage have a conflict between mindependency and what local azure-identity requires. resolve it (#20391) * [AutoRelease] t2-costmanagement-2021-07-27-84467 (#19949) * CodeGen from PR 15291 in Azure/azure-rest-api-specs Updating Query filter description (#15291) Co-authored-by: Jorge Chavez Nieto <[email protected]> * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Jorge Chavez Nieto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * [AutoRelease] t2-servicebus-2021-08-19-00756 (#20350) * CodeGen from PR 14749 in Azure/azure-rest-api-specs add apitestErrorCode doc (#14749) * add apitestErrorCode doc * add armRPC doc * add roundTripInconsistentProperty doc * small fix * update * update signalR armTemplate * update yaml * new file * add testScenario with armTemplate * update doc * update readme.md * update doc * update NOTE section * add signalRCreateOrUpdate example file * udpate doc * add generate test scenario section * update doc * add serviceFacbric test scenario file * update managedClusters.yaml * update generateABasicTestScenario.md * update features doc * add run api test gif * version,CHANGELOG * test * Update test_cli_mgmt_servicebus_namespace.py * Update test_queues.py * Update test_queues.py Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: Zed Lei <[email protected]> Co-authored-by: Jiefeng Chen <[email protected]> * [AutoRelease] t2-policyinsights-2021-08-23-78735 (#20381) * CodeGen from PR 15165 in Azure/azure-rest-api-specs [policyinsights] Fix next_link (#15165) * fix next_link * directive * Update readme.nodejs.md * js directvie * Update readme.md * supress other language * Update readme.go.md * swagger lint * swagger lint * swagger lint * Update readme.md * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: Zed Lei <[email protected]> * [AutoRelease] t2-datamigration-2021-08-19-04035 (#20349) * CodeGen from PR 15259 in Azure/azure-rest-api-specs Dev datamigration microsoft.data migration 2021 06 30 (#15259) * Created new api version and refactoring changes (such as fixing new build restrictions) * Adding breaking changes * Adding location to the services update example * Adding location for projects update and refactoring the task command * Resolving tasks command ModelValidation test failure * Resolving issue based on zhenglaizhang's recommendation * Adding comma to resolve issue Co-authored-by: Artem Pavlichenko <[email protected]> * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Artem Pavlichenko <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * Chain exceptions from LibsecretPersistence (#20380) * [translation] fix poller.details (#20392) * fixing docstring types for async client * populate poller.details.id immediately * pylint * add bug fix to changelog * improve begin_translation docstring (#20401) * Update .docsettings.yml (#20402) * Sync eng/common directory with azure-sdk-tools for PR 1918 (#20342) * Stress test deployment script resiliency fixes * Use PSModule-Helpers to install powershell-yaml Co-authored-by: Ben Broderick Phillips <[email protected]> * improve docs and samples for glossaries and custom models (#18587) * update the readme * update readme file * added custom translation samples * fix 'no-locale' thing in links * update glossary docs * update glossaries * link to sample glossaries instead of writing code in readme * update custom model sample linking * remove relative linking in readme * make subheadings in bold text to be more readable * conform with 'Document Translation' naming * disambiguate container sas url * capitaliz Azure name * remove misplaced period * update samples -> custom model * update async sample -> custom model * remove localization from url * update readme with new file types for glossaries * adding sample glossaries -> xlf * white space * use simplified single input method * update 'job' terminology * update azure-core naming * update glossary blob file reference name * link to supported glossaries table * remove locale from url * [purview account] first release for azure-purview-account (#20335) * purview account * CI * verion-tolerant * endpoint order * add test for collections * Update CHANGELOG.md * review * update reamdme * skip custom translation model samples (#20414) * Address API view issues (#20407) * logs client * lc-2 * models changes * more changes * more changes * more arch changes * changelog * tests * lint * fix tests * timespan * [translation] naming alignment (#20406) * list_all_translation_statuses -> list_translation_statuses * list_all_document_statuses -> list_document_statuses * cancelled -> canceled, cancelling -> canceling * FileFormat -> DocumentTranslationFileFormat * update changelog * make core tests offline (#19986) * make core tests offline * clean up * use localhost instead of 127.0.0.1 * update * update * update * Get rid of LogsBatchQueryResult (#20418) * Get rid of LogsBatchQueryResult * Update sdk/monitor/azure-monitor-query/README.md * oops * comments * [AutoRelease] t2-iothub-2021-08-25-25696 (#20409) * CodeGen from PR 15722 in Azure/azure-rest-api-specs Removing readonly for isVerified property as it is no more readonly property (#15722) * version,CHANGELOG * test Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * [AutoRelease] t2-healthcareapis-2021-08-26-27542 (#20422) * CodeGen from PR 15381 in Azure/azure-rest-api-specs Update readme.md (#15381) * version,CHANGELOG Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * Stip empty changelog sections before release (#20437) Co-authored-by: Chidozie Ononiwu <[email protected]> * Update question-answering readme links (#20439) * ensure test principal creation succeeds properly (#20446) Co-authored-by: scbedd <[email protected]> * New dummy password in README (#20328) * Sync eng/common directory with azure-sdk-tools for PR 1943 (#20450) * Bug fix: Storing .env file for bicep file at service dir * Styling Co-authored-by: Albert Cheng <[email protected]> * [AutoRelease] t2-keyvault-2021-08-26-55443 (#20423) * CodeGen from PR 15458 in Azure/azure-rest-api-specs Add rotation permission (#15458) * Add rotation permission * Update keys.json Remove required from lifetimeActions * Change to expiryTime * version,CHANGELOG * test Co-authored-by: SDKAuto <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> * new recordings Co-authored-by: SDKAuto <[email protected]> Co-authored-by: Audunn Baldvinsson <[email protected]> Co-authored-by: PythonSdkPipelines <PythonSdkPipelines> Co-authored-by: iscai-msft <[email protected]> Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Rakshith Bhyravabhotla <[email protected]> Co-authored-by: swathipil <[email protected]> Co-authored-by: McCoy Patiño <[email protected]> Co-authored-by: Adam Ling (MSFT) <[email protected]> Co-authored-by: Tong Xu (MSFT) <[email protected]> Co-authored-by: ckairen <[email protected]> Co-authored-by: Charles Lowell <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: Azure CLI Bot <[email protected]> Co-authored-by: Jorge Chavez Nieto <[email protected]> Co-authored-by: Zed Lei <[email protected]> Co-authored-by: Jiefeng Chen <[email protected]> Co-authored-by: Artem Pavlichenko <[email protected]> Co-authored-by: Krista Pratico <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Mohamed Shaban <[email protected]> Co-authored-by: msyyc <[email protected]> Co-authored-by: Xiang Yan <[email protected]> Co-authored-by: Chidozie Ononiwu <[email protected]> Co-authored-by: annatisch <[email protected]> Co-authored-by: Albert Cheng <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Addressing the bug in which the release pipeline's smoke test doesn't download the correct package version that was released.
Pipeline Reference after the fix:
Downloaded Version:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1052595&view=logs&j=811de798-cac6-5a82-8dac-ad597ae5b1be&t=70ccc55a-509b-52ea-a60f-0b83826d1abf&l=183&l=183
Retry Attempt:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1052595&view=logs&j=811de798-cac6-5a82-8dac-ad597ae5b1be&t=70ccc55a-509b-52ea-a60f-0b83826d1abf&l=183&l=166
Published Version:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1052595&view=logs&j=8a59aa26-f1cb-5dea-999b-76ab4b63e138&t=2a0c1ff9-8f40-5875-8d33-2fedc244c604&l=140
Previous issue of version mismatch:
Downloaded Version:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1045637&view=logs&j=811de798-cac6-5a82-8dac-ad597ae5b1be&t=299c6ba1-91d4-5571-bc9b-ca88aacc6166&l=133
Published Version:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1045637&view=logs&j=8a59aa26-f1cb-5dea-999b-76ab4b63e138&t=2a0c1ff9-8f40-5875-8d33-2fedc244c604&l=140